-
Notifications
You must be signed in to change notification settings - Fork 494
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ENH: Add API to select items in the DICOM browser #1002
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Consider rebasing the topic so that we avoid commit like |
No specific reason, it was just what I had in my build tree. Thanks for the review and comments, I'll rebase on latest master. |
4279893
to
f4b9096
Compare
d451d05
to
d85ce3c
Compare
176c887
to
8894116
Compare
This allows selecting patient/study/series in the DICOM browser GUI. Co-authored-by: Jean-Christophe Fillion-Robin <[email protected]>
8894116
to
4bd6926
Compare
@jcfr I've updated ctkDICOMTableView::setCurrentSelection to use |
@jcfr Thanks for your comments, I addressed all of them, let me know if this is good to go now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating 🙏
Once this is integrated, I will then rebase and update #1005
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR commontk#1002 (ENH: Add API to select items in the DICOM browser)
This commit updates the implementation to account for changes introduced in PR #1002 (ENH: Add API to select items in the DICOM browser)
Allow selecting patient/study/series in the DICOM browser GUI.
This is can be used for example for highlighting new items after DICOM import.